Open
Conversation
Co-authored-by: RubenCerna2079 <32799214+RubenCerna2079@users.noreply.github.com>
Co-authored-by: RubenCerna2079 <32799214+RubenCerna2079@users.noreply.github.com>
Co-authored-by: RubenCerna2079 <32799214+RubenCerna2079@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add CLI command dab autoentities configure
Add CLI command Jan 30, 2026
dab autoentities-configure
RubenCerna2079
requested changes
Feb 5, 2026
- Fix GraphQL template to only serialize 'enabled' property (not 'type') - Improve error messages to show all valid parameter values - Use EnumExtensions.GenerateMessageForInvalidInput for EntityCacheLevel - Update MCP dml-tool error message to show valid values Co-authored-by: RubenCerna2079 <32799214+RubenCerna2079@users.noreply.github.com>
RubenCerna2079
requested changes
Feb 6, 2026
- Require permissions when creating a new autoentity definition - Allow updating existing definitions without providing permissions - Improve error handling to distinguish between parsing failure and missing permissions Co-authored-by: RubenCerna2079 <32799214+RubenCerna2079@users.noreply.github.com>
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
JerryNixon
approved these changes
Feb 16, 2026
Contributor
There was a problem hiding this comment.
Please rename autoentities-configure to auto-config
- [Verb("auto-config", ...)] in AutoentitiesConfigureOptions.cs
- Consider renaming the class to AutoConfigOptions
- Update test class name, log messages, and help text
Future
Almost certainly, we will call this feature "auto config" from here on.
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
dab autoentities-configuredab auto-config
Aniruddh25
reviewed
Feb 19, 2026
Aniruddh25
reviewed
Feb 19, 2026
Aniruddh25
reviewed
Feb 19, 2026
Aniruddh25
reviewed
Feb 19, 2026
Aniruddh25
requested changes
Feb 19, 2026
Collaborator
Aniruddh25
left a comment
There was a problem hiding this comment.
Clarify if we want caching to be enabled by default or disabled for autoentities. Runtime has it disabled by default, so makes sense to keep it disabled.
Also, how do we populate autoentities section after the init command?
Contributor
|
|
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
Contributor
|
/azp run |
|
Azure Pipelines successfully started running 6 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why make this change?
dab auto-config#2964Implements CLI command to configure autoentities definitions via command line. Previously, users had to manually edit the config file to add or modify autoentities entries.
What is this change?
Added
auto-configcommand that upserts autoentities definitions with support for:--patterns.include,--patterns.exclude,--patterns.name)role:actionsformatImplementation:
AutoConfigOptions.cs- Command options class following existing CLI patternsConfigGenerator.TryConfigureAutoentities()- Main handler with builder methods for patterns, template, and permissionsProgram.csparserBug fixes:
AutoentityConverterandAutoentityTemplateConverter- Added missing MCP options serialization logic that prevented MCP settings from persisting to config fileAutoentityTemplateConverter- Fixed GraphQL template serialization to only writeenabledproperty, excludingtypeobject (singular/plural) which is determined by generated entitiesImprovements:
EnumExtensions.GenerateMessageForInvalidInput<T>()pattern--permissionsonly when creating a new autoentity definition. When updating an existing definition, permissions are optional and preserved from the existing configuration.How was this tested?
Added 8 unit tests covering:
Sample Request(s)
Result in config:
{ "autoentities": { "my-def": { "patterns": { "include": ["dbo.%", "sys.%"], "exclude": ["dbo.internal%"], "name": "{schema}_{table}" }, "template": { "rest": { "enabled": true }, "graphql": { "enabled": true }, "mcp": false, "cache": { "enabled": true, "ttl-seconds": 60, "level": "l1l2" } }, "permissions": [ { "role": "anonymous", "actions": [ { "action": "read" } ] } ] } } }Notes:
"graphql": { "enabled": true }without the emptytypeobject that was previously being written.Original prompt